Skip to content

Conversation

@etra0
Copy link
Contributor

@etra0 etra0 commented Mar 12, 2023

This would be the initial PR to be able to compile stuff in Windows.

In particular, MSVC is very picky about the features you can use and you cannot.

With C++11

  • You cannot use designated initializers (when initializing a struct, you cannot specify the fields names)
  • You cannot use VLAs, so I changed it to a vector.

A PR for the CMake build system (as agreed in #22) will be separated.

These changes were tested with MSVC 19.34.31937.0 (VS Studio 2022) and in macOS 12.6 with Apple clang version 13.1.6

@ggerganov ggerganov mentioned this pull request Mar 12, 2023
@etra0 etra0 requested a review from ggerganov March 12, 2023 15:47
@ggerganov ggerganov merged commit eb062bb into ggml-org:master Mar 12, 2023
Hades32 pushed a commit to Hades32/llama.cpp that referenced this pull request Mar 21, 2023
theo77186 pushed a commit to theo77186/llama.cpp that referenced this pull request Oct 28, 2025
jesusmb1995 pushed a commit to jesusmb1995/llama.cpp that referenced this pull request Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants